Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

require any hyphenation to be included in the parquet prefix #842

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

require any hyphenation to be included in the parquet prefix #842

wants to merge 1 commit into from

Conversation

jameshalgren
Copy link
Contributor

In the t-route configuration file, to meet the requirements for input to the TEEHR evaluation system, the ids need to be tagged according to their hydrofabric element type. The prefix_ids element allows for this to be set.

This PR allows the prefix to be completely arbitrary, removing the hard-coded hyphen '-'.

The suggested output change to be added to the end of the yaml file after the stream_output section is now:

  parquet_output:
    parquet_output_folder: outputs/troute/
    configuration: short_range
    prefix_ids: wb-

ping @mgdenno @karnesh

@jameshalgren
Copy link
Contributor Author

Note:
The output folder in the example .yaml input is configured as found in the standard file path for NextGen-based t-route simulations generated with the NGIAB_Data_process application and simulation with the NGIAB_CloudInfra package.
https://github.com/CIROH-UA/NGIAB_data_preprocess
and
https://github.com/CIROH-UA/NGIAB-CloudInfra,
respectively.

allow non-hyphenated prefix
6b19d67 
Also means that if you want a hyphen, it must be included.
See example in LowerColorado_TX/test_AnA_V4_NHD.yaml
@jameshalgren jameshalgren force-pushed the remove_hyphen_from_parquet_prefix branch from e70b09e to 6b19d67 Compare September 5, 2024 12:58
@AminTorabi-NOAA AminTorabi-NOAA mentioned this pull request Sep 10, 2024
Open
21 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jameshalgren